Skip to content

deploy: replace manual bridge restart hint with baudbot restart#103

Merged
benvinegar merged 1 commit into
mainfrom
bentlegen/deploy-restart-hint
Feb 21, 2026
Merged

deploy: replace manual bridge restart hint with baudbot restart#103
benvinegar merged 1 commit into
mainfrom
bentlegen/deploy-restart-hint

Conversation

@benvinegar
Copy link
Copy Markdown
Member

@benvinegar benvinegar commented Feb 21, 2026 •
edited
Loading

Summary

  • remove the manual node bridge.mjs restart hint from deploy output
  • replace with the managed restart command: sudo baudbot restart

Why

  • avoids suggesting a direct bridge invocation that can bypass managed runtime behavior
  • matches current start/restart flow where launcher selects the correct bridge mode

Validation

  • bash -n bin/deploy.sh

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Feb 21, 2026

Greptile Summary

Updated deploy script output to recommend sudo baudbot restart instead of manually running node bridge.mjs. This aligns with the managed service approach and ensures the launcher selects the correct bridge mode.

  • Replaced manual bridge restart hint with managed restart command
  • Improves consistency with the systemd-based lifecycle management
  • Note: SECURITY.md:138 still contains the old manual command and could be updated for consistency

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • Simple, well-scoped change that improves consistency. Replaces a manual command hint with the proper managed restart command. The change is non-breaking and only affects console output.
  • No files require special attention

Important Files Changed

Filename Overview
bin/deploy.sh Updated restart hint to use managed baudbot restart command instead of direct bridge invocation

Last reviewed commit: d6ed0f2

greptile-apps[bot]
greptile-apps Bot reviewed Feb 21, 2026
View reviewed changes
Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread bin/deploy.sh
Comment on lines +451 to +452
echo "Restart runtime services to load changes (recommended):"
echo " sudo baudbot restart"
Copy link
Copy Markdown

@greptile-apps greptile-apps Bot Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SECURITY.md:138 still references the old manual bridge restart command. Consider updating it for consistency.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Prompt To Fix With AI 
This is a comment left during a code review.
Path: bin/deploy.sh
Line: 451-452

Comment:
`SECURITY.md:138` still references the old manual bridge restart command. Consider updating it for consistency.

<sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub>

How can I resolve this? If you propose a fix, please make it concise.

@benvinegar benvinegar merged commit 3801c32 into main Feb 21, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@benvinegar